Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX CMDER_ROOT for admin launch #311

Merged
merged 1 commit into from
Mar 18, 2015

Conversation

sescandell
Copy link
Contributor

When launching a cmd in admin mode, if %ConEmuDir%\..\.. contains any space, it breaks the init script.
Thanks to this patch, the path is correctly defined now.

When launching a cmd in admin mode, if %ConEmuDir%\..\.. contains any space, it breaks the scripts.
Thanks to this patch, the path is correctly defined now.
@killua99
Copy link

Maybe you don't need this comment but. Applying this change works with externals tools like PhpStorm when I setup the console with cmder

MartiUK added a commit that referenced this pull request Mar 18, 2015
@MartiUK MartiUK merged commit 7169f49 into cmderdev:development Mar 18, 2015
@FrankPeters
Copy link

@killua99 Could you explain how you added Cmder to PhpStorm to use as Terminal? When I use C:\cmder\Cmder.exe as the Shell path, the Terminal window opens and then closes but Cmder opens up in a separate window (but not in the correct directory).

@sescandell sescandell deleted the fix-adminlaunch branch April 2, 2015 07:35
@killua99
Copy link

killua99 commented Apr 7, 2015

Hi, I was on holidays I forgot to reply. I set on the terminal "Shell path"

cmd /k "C:\opt\cmder\vendor\init.bat"

Hope it work for you.

@FrankPeters
Copy link

Hey @killua99 it worked however the prompt is pretty awkward, looks corrupt or something. Well, I'll just continue with a separate instance on my 2nd monitor then, thanks for the reply! 👍

@killua99
Copy link

killua99 commented Apr 8, 2015

You must to be sure you had this patch in your init.bat file. If not it look broken

@FrankPeters
Copy link

Oh I see, it's not yet in stable release. Great, then I'll try it again when it's merged into master. Thanks!

@zerosurfer
Copy link

@killua99 I have the same issue , it looks awkward.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants